Skip to content

EX-316: Create user settings entity with default settings and ISO validations#88

Open
hptrk wants to merge 2 commits intodevelopfrom
feature/EX-316-user-settings
Open

EX-316: Create user settings entity with default settings and ISO validations#88
hptrk wants to merge 2 commits intodevelopfrom
feature/EX-316-user-settings

Conversation

@hptrk
Copy link
Copy Markdown
Owner

@hptrk hptrk commented Mar 23, 2026

előző PR-ből ágaztam le, elég a commitot nézni

@hptrk hptrk requested review from ntamasa and tamibalogh March 23, 2026 23:01
@hptrk hptrk self-assigned this Mar 23, 2026
@hptrk hptrk added enhancement New feature or request backend labels Mar 23, 2026
@hptrk hptrk changed the title feature/EX-316-user-settings EX-316: Create user settings entity with default settings and ISO validations Mar 23, 2026
@ntamasa ntamasa added this to the v1.2.0 milestone Mar 24, 2026
@ntamasa
Copy link
Copy Markdown
Collaborator

ntamasa commented Mar 24, 2026

ezt majd mikor kész az 1.1.0 mergeljük

Copy link
Copy Markdown
Collaborator

@ntamasa ntamasa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jelenleg ez csak a currencyt változtatja, ezzel nem tudunk majd normálisan statisztikát megjeleníteni, ha valaki átrakja euróra mert nyaral pl. akkor abban logol, hogy 4€, 15€, akkor a DB-ben jelenleg 4 és 15 lesz az elmentett érték, ekkor ez a statisztikát megbassza. Lehet egy ideiglenes megoldásként az adminnak lehetne lehetősége adott currency-khez ft árfolyamot állítgatni, mint mondjuk itt is a sparban meg itt ott ki van írva, hogy ha euróval fizetsz ennyi lesz. Ezt esetleg a profil fülön lehetne látni, hogy ha nem forint megjelenítünk egy kis info boxot amiben leírjuk, hogy ez a váltás. Ezzel annyi porbléma lenne, hogy nem HUF transaction váltáskor az aktuális értékre átváltanánk és csak forintban tárolnánk az értékeket a DB-ben. Így ezzel meg lenne oldva az árfolyam alakulásából adódó folyamatos értékváltozás is. Szerintem nem rossz, esetleg később lehet dinamikusan X időnként lekérni valami külső servicetől az árfolyamot, mondjuk minden nap éjfélkor vagy valami hasonló. De egyelőre szerintem jó megoldás lehet.
Az is lehet hogy azt nem ebben PR-ben csináljuk, hanem külön tickettel, külön PR-ban.

@ntamasa ntamasa self-requested a review March 28, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants